-
Notifications
You must be signed in to change notification settings - Fork 14
fix: Merge IexecERC20Common & IexecERC20
#321
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
… IexecERC20 to include events directly
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## chore/solidity-v8 #321 +/- ##
==================================================
Coverage 96.36% 96.36%
==================================================
Files 34 33 -1
Lines 1127 1127
Branches 227 213 -14
==================================================
Hits 1086 1086
Misses 41 41 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
… use abstract FacetBase
…aceNative to FacetBase and IexecERC20Facet to IexecERC20Base, while consolidating related interfaces
| new IexecCategoryManagerFacet__factory(), | ||
| new IexecConfigurationExtraFacet__factory(), | ||
| new IexecConfigurationFacet__factory(iexecLibOrders), | ||
| new IexecERC20Facet__factory(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a todo : unplug this facet from proxy = 0xB0152eC6f48E64a92B66D4736aFA1b02d8D45169#code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix Commit: bc392fb
I created the future migration script
… in Solidity documentation
No description provided.